Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Additional query to get jobs by ids #2161

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

hyper987654321
Copy link

The only current way to get jobs by ids is to get chunks of jobs with ScheduledJobs(from, to) function, which is time-consuming if you have many of them.
This PR tries to solve this issue by providing directly a function that lets you get specific jobs that you need.

Change parameter type, introduce exception for 0 length

Simplify get by id function

Remove gap
@hyper987654321 hyper987654321 force-pushed the Additional-query-to-get-jobs-by-id branch from e4e811b to a1ad060 Compare February 3, 2023 12:40
@burningice2866
Copy link
Contributor

Auch, thats going to break all existing 3rd-party storage providers.

@odinserj odinserj added this to the Hangfire 2.0.0 milestone Feb 16, 2023
@odinserj
Copy link
Member

You are correct Pauli, unfortunately can't merge it to 1.X version due to breaking changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants